Add similarweb extension#27452
Conversation
|
Congratulations on your new Raycast extension! 🚀 We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days. Once the PR is approved and merged, the extension will be available on our Store. |
Greptile SummaryThis PR adds a new Similarweb extension with two commands: Show Website Data (fetches analytics for a typed domain or the active browser tab) and History (browses locally stored snapshots). The implementation is clean — domain parsing, LocalStorage persistence, Markdown rendering, and cancellation guards are all well-handled. The one remaining style note is that Confidence Score: 5/5Safe to merge; the only remaining finding is a P2 locale style issue in timestamp formatting. All P1 issues from prior review rounds have been addressed. The single new finding is a P2 style note (toLocaleString() without locale), which does not affect correctness or safety. No files require special attention beyond the one-line formatTimestamp fix in src/lib/format.ts. Important Files Changed
Prompt To Fix All With AIThis is a comment left during a code review.
Path: extensions/similarweb/src/lib/format.ts
Line: 721-723
Comment:
**`toLocaleString()` uses the system locale**
`new Date(value).toLocaleString()` formats the timestamp using the OS locale, so users with non-English system settings will see dates in their local language. Raycast only supports US English; passing `"en-US"` keeps output consistent across all machines.
```suggestion
function formatTimestamp(value: string): string {
return new Date(value).toLocaleString("en-US");
}
```
**Rule Used:** What: Do not implement custom localization logic i... ([source](https://app.greptile.com/review/custom-context?memory=78677f74-64f7-483b-b90d-9c33e2e049da))
How can I resolve this? If you propose a fix, please make it concise.Reviews (5): Last reviewed commit: "chore(similarweb): update dependencies a..." | Re-trigger Greptile |
- Bump devDependencies to latest versions for better performance and compatibility. - Add DOM library to TypeScript configuration. - Introduce eslint configuration file for improved linting setup.
0xdhrv
left a comment
There was a problem hiding this comment.
Looks good to me, approved ✅
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
Description
Fetch Similarweb analytics, which include rankings, traffic trends, and audience insights for any website, and explore the locally stored history of your lookups.
Commads/Features
demo.yafw.balanced.mp4
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder